Fix TrainingModule class declaration formatting#19830
Conversation
Summary: Reformat `TrainingModule` class declaration to place `private constructor` on the same line as the class name, with `Closeable` wrapping to the next line. This follows the standard Kotlin formatting convention. Differential Revision: D106574405
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/19830
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ⏳ No Failures, 19 PendingAs of commit 6e9cf5b with merge base d366f43 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
meta-cla
Bot
added
the
CLA Signed
|
@kirklandsign has exported this pull request. If you are a Meta employee, you can view the originating Diff in D106574405. |
This PR needs a
|
There was a problem hiding this comment.
Pull request overview
This PR adjusts Kotlin formatting for the TrainingModule class declaration in the Android training wrapper to better match standard Kotlin style (placing the private constructor on the same line as the class name and wrapping the Closeable supertype onto the next line).
Changes:
- Reformats the
TrainingModuleclass header to keepclass ... private constructor(...)on one line. - Wraps the
: Closeableportion onto a continuation line with consistent indentation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary: Reformat
TrainingModuleclass declaration to placeprivate constructoron the same line as the class name, withCloseablewrapping to the next line. This follows the standard Kotlin formatting convention.Differential Revision: D106574405